Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FrontEnd]enable PDPD 16 ops conversion Part2 #6632

Merged
merged 13 commits into from
Jul 30, 2021

Conversation

zhangYiIntel
Copy link
Contributor

@zhangYiIntel zhangYiIntel commented Jul 14, 2021

Details:

Tickets:

  • 53052

@zhangYiIntel zhangYiIntel added the category: Core OpenVINO Core (aka ngraph) label Jul 14, 2021
@zhangYiIntel zhangYiIntel added this to the 2022.1 milestone Jul 14, 2021
@zhangYiIntel zhangYiIntel requested review from ilyachur, mvafin, jane-intel, nosovmik and a team July 14, 2021 02:38
@zhangYiIntel zhangYiIntel changed the title [FrontEnd]enable 16 ops conversion [FrontEnd]enable PDPD 16 ops conversion Jul 14, 2021
@zhangYiIntel zhangYiIntel changed the title [FrontEnd]enable PDPD 16 ops conversion [FrontEnd]enable PDPD 16 ops conversion Part2 Jul 14, 2021
@ilyachur
Copy link
Contributor

@jane-intel @mvafin Could you have a look?

@nosovmik
Copy link
Contributor

Any reason of not including test models here? Just wanted to check if all cases are covered by tests

@zhangYiIntel
Copy link
Contributor Author

Hi @nosovmik The tests cases are removed here because previously, we have different opinions about how to test conversion. Therefore we remove them in this PR and test them locally. I think we can add them back in another PR because we have alignment with Ilya about how to test the conversion.

* fix paddle conversion

* ops forward declaration
@zhangYiIntel zhangYiIntel requested a review from a team July 27, 2021 02:18
@mvafin mvafin requested a review from mangguo321 July 27, 2021 13:44
@zhangYiIntel zhangYiIntel requested a review from jane-intel July 29, 2021 08:32
@azhogov azhogov merged commit 81377a7 into openvinotoolkit:master Jul 30, 2021
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
* [FrontEnd]enable 16 ops conversion

* [FrontEnd]fix typo in interpolate

* param support tensor (#3)

* Update hard_sigmoid.cpp (#4)

* Yi3/fix paddle part2

* fix paddle conversion

* ops forward declaration

* use tile instead of broadcast

* Add validation check in dropout (#8)

* fix conflict

* fix code style

* fix transpose2

* use perm size in transpose2

* remove check in transpose2

Co-authored-by: Luo Cheng <[email protected]>
Co-authored-by: Mang Guo <[email protected]>
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
* [FrontEnd]enable 16 ops conversion

* [FrontEnd]fix typo in interpolate

* param support tensor (#3)

* Update hard_sigmoid.cpp (#4)

* Yi3/fix paddle part2

* fix paddle conversion

* ops forward declaration

* use tile instead of broadcast

* Add validation check in dropout (#8)

* fix conflict

* fix code style

* fix transpose2

* use perm size in transpose2

* remove check in transpose2

Co-authored-by: Luo Cheng <[email protected]>
Co-authored-by: Mang Guo <[email protected]>
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
* [FrontEnd]enable 16 ops conversion

* [FrontEnd]fix typo in interpolate

* param support tensor (#3)

* Update hard_sigmoid.cpp (#4)

* Yi3/fix paddle part2

* fix paddle conversion

* ops forward declaration

* use tile instead of broadcast

* Add validation check in dropout (#8)

* fix conflict

* fix code style

* fix transpose2

* use perm size in transpose2

* remove check in transpose2

Co-authored-by: Luo Cheng <[email protected]>
Co-authored-by: Mang Guo <[email protected]>
@zhangYiIntel zhangYiIntel deleted the yi3/enable_pdpd_ops branch July 25, 2024 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants